-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement Entry IsSpellCheckEnabled property #7442
Conversation
@jsuarezruiz any idea why tests are failing ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just the API breaks are gonna be fun.
@Redth will we need to bump this to net7 ? Other option is doing a 6.1.0 |
Added the .NET 7 milestone. |
@jsuarezruiz can you add this to the API ? |
@jsuarezruiz can you rebase this one ? i might have screwed it up |
To help focus and move faster, we're going to have fewer open PRs and focus on the ones we'll be working on in each release. This PR is not included in the next release so I close it for now. I will reopen it and adapt to changes later. |
🚨 API change(s) detected @davidbritch FYI |
Thank you for your pull request. We are auto-formatting your source code to follow our code guidelines. |
@@ -154,6 +154,11 @@ public static void UpdateIsTextPredictionEnabled(this TextBox textBox, ITextInpu | |||
textBox.UpdateInputScope(textInput); | |||
} | |||
|
|||
public static void UpdateIsSpellCheckEnabled(this TextBox textBox, ITextInput textInput) | |||
{ | |||
textBox.UpdateInputScope(textInput); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It appears that this method does not set the spell check or the text prediction outside of a custom keyboard.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not using a custom keyboard, seems to be setting the text prediction here https://github.com/dotnet/maui/blob/main/src/Core/src/Platform/Windows/TextBoxExtensions.cs#L171
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor nit, but I think using HasFlag will make the coder ore readable and accessible to more jr devs.
This was reverted in #14147 |
Description of Change
Implement Entry
IsSpellCheckEnabled
property.Fixes (but reverted) #8558